Skip to content

Conversation

@rpanic
Copy link
Member

@rpanic rpanic commented Nov 9, 2024

This PR now fixes the remaining issues that led to errors during settlement with proofs enabled.

  • Added SettlementStartupModule to handle the compiling of Settlement Contract and returning of the VKs of those contracts
  • Used the returned VKs in the deployments on the Baselayer
  • Fixed serialization issues with the previousProofs value on SC calls
  • Fixed serialization issues on AccountUpdateForest usage as contract arguments
  • Made LocalBlockchain respect protokits AreProofsEnabled

This PR is the last of a PR chain in order of:
#210
#218
#219
#220
#224 (this)

Closes #226
Closes #207

# Conflicts:
#	packages/common/src/utils.ts
#	packages/protocol/src/prover/block/BlockProver.ts
#	packages/protocol/src/prover/statetransition/StateTransitionProvable.ts
#	packages/protocol/src/prover/statetransition/StateTransitionProver.ts
#	packages/sequencer/src/protocol/production/tasks/StateTransitionTask.ts
@rpanic rpanic changed the base branch from feature/contract-compiling to develop January 28, 2025 12:32
# Conflicts:
#	packages/sequencer/src/protocol/production/BlockTaskFlowService.ts
#	packages/sequencer/src/protocol/production/tasks/StateTransitionTask.ts
@maht0rz maht0rz merged commit c0b98ce into develop Jan 28, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable Proving Allow workers to consume runtime values at startup

3 participants